Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move HDF4 tests to their own directory #895

Merged
merged 9 commits into from
Mar 8, 2018

Conversation

edhartnett
Copy link
Contributor

In this PR I move the HDF4 tests into directory hdf4_test.

This also includes the contents of PR #888 in order to pass my CI.

Part of #870.
Fixes #869.
Fixes #744.
Part of #177.

@WardF
Copy link
Member

WardF commented Mar 5, 2018

Looks good; I assume the single failure was a one-off. Restarted it. We are at a retreat today, but I'm working my way backwards and want to get a 4.6.1 release out ASAP.

@WardF WardF added this to the 4.6.1 milestone Mar 5, 2018
@WardF WardF self-assigned this Mar 5, 2018
@edhartnett
Copy link
Contributor Author

@WardF the failure was due to unrelated issue #896. Restarting should work just fine, but you must do a make clean on the workspace, or just wipe out the workspace.

@WardF
Copy link
Member

WardF commented Mar 5, 2018

The failure was a single line item out of the various matrices, and appears to have been a timeout pulling down the docker image. A nice side effect of the docker tests, fwiw, is that it is always a 100% clean workspace every time it is run :).

@WardF WardF merged commit f6a0366 into Unidata:master Mar 8, 2018
@edhartnett edhartnett deleted the ejh_hdf4_more branch March 8, 2018 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants